Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix Configuring Custom Dokka Plugin #3870

Conversation

solonovamax
Copy link

@solonovamax solonovamax commented Oct 17, 2024

Fixes #3869

  • Register binding for DokkaPluginParametersBuilder
  • Remove @Internal annotation on jsonEncode() for DokkaPluginParametersBuilder (see [Bug] Cannot Configure Custom Dokka Plugin #3869)
  • Annotate the getter for DokkaPluginParametersBuilder.objects instead of the field (the annotation only works for getters)
  • Add missing functions for adding files, directories, and maps, which were implied but not present for DokkaPluginParametersBuilder
  • remove overriden DokkaPluginParametersBuilder.pluginFqn property as it just caused issues
  • Add documentation for configuring custom plugins in dokka-migration.md

- Register binding for `DokkaPluginParametersBuilder`
- Remove @internal annotation on `jsonEncode()` for `DokkaPluginParametersBuilder`
- Annotate the getter for `DokkaPluginParametersBuilder.objects` (the annotation only works for getters)
- Add missing functions for adding files, directories, and maps, which were implied but not present for `DokkaPluginParametersBuilder`
- remove overriden `DokkaPluginParametersBuilder.pluginFqn` property as it just caused issues
- Add documentation for configuring custom plugins in dokka-migration.md

Signed-off-by: solonovamax <[email protected]>
@adam-enko adam-enko self-assigned this Oct 17, 2024
@adam-enko adam-enko added the runner: Gradle plugin An issue/PR related to Dokka's Gradle plugin label Oct 17, 2024
@adam-enko adam-enko marked this pull request as draft October 17, 2024 07:41
@adam-enko
Copy link
Member

Thank you for the approach, it helped me investigate the issue further.

However, in addition to the problems you found, I found more issues, so we're going to remove it from DGP (see #3869 (comment)).

Thank you again for your efforts!

@adam-enko adam-enko closed this Oct 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
runner: Gradle plugin An issue/PR related to Dokka's Gradle plugin
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug] Cannot Configure Custom Dokka Plugin
2 participants